Conversation
- Use declarative `normalizes` API instead of before_validation callbacks
- Name: remove all whitespace ("홍 길 동" → "홍길동")
- Phone: remove all non-digit characters ("010-1234-5678" → "01012345678")
- Update TECHSPEC §7.1 to reflect normalizes approach
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Add Course#remaining_slots to calculate available capacity - Add home page showing available courses (capacity > 0) with remaining slots - Add registration form with required fields (name, phone, birth_date, gender, address) - Add routes for home and nested registrations under courses - Add testing guideline: update old tests when model methods replace inline logic Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…vation Add create action to RegistrationsController with validation error handling. Auto-set race_id from course via before_validation callback. Display validation errors at top of form and preserve user input on resubmission. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add maximum length constraint for registration name field as specified in TECHSPEC. Validates after normalization (space removal). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add HTML maxlength=10 to name text field for client-side UX guard. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add maximum length constraint for registration address field with server-side validation and HTML maxlength attribute. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add maximum length constraint for phone_number field. Server validates 11 digits (post-normalization), HTML allows 13 chars to accommodate dashes in user input. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.